Skip to content

Conversation

killme2008
Copy link
Contributor

@killme2008 killme2008 commented Sep 25, 2025

What's Changed in this PR

Describe the change in this PR

Close #2144

The k8s cluster needs to support the permission mode too. cc @zyy17

Checklist

  • Please confirm that all corresponding versions of the documents have been revised.
  • Please ensure that the content in sidebars.ts matches the current document structure when you changed the document structure.
  • This change requires follow-up update in localized docs.

Signed-off-by: Dennis Zhuang <[email protected]>
@killme2008 killme2008 requested review from Copilot and nicecui and removed request for fengjiachun and nicecui September 25, 2025 11:34
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds comprehensive documentation for user permission management in GreptimeDB's static authentication, expanding beyond basic username/password configuration to include role-based access control.

Key changes:

  • Adds permission mode configuration (readonly, writeonly, readwrite) with syntax examples
  • Documents dynamic file reloading capability using watch_file_user_provider
  • Provides detailed configuration examples with mixed permission scenarios

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
docs/user-guide/deployments-administration/authentication/static.md English documentation with permission modes, dynamic reloading, and comprehensive examples
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/deployments-administration/authentication/static.md Chinese translation of the same permission documentation updates

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

cloudflare-workers-and-pages bot commented Sep 25, 2025

Deploying greptime-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: c0b14e3
Status: ✅  Deploy successful!
Preview URL: https://4d7f39e7.greptime-docs.pages.dev
Branch Preview URL: https://feature-static-user-permissi.greptime-docs.pages.dev

View logs

Signed-off-by: Dennis Zhuang <[email protected]>
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@shuiyisong shuiyisong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

killme2008 and others added 7 commits September 26, 2025 14:37
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Dennis Zhuang <[email protected]>
Signed-off-by: Dennis Zhuang <[email protected]>
Signed-off-by: Dennis Zhuang <[email protected]>
@killme2008 killme2008 merged commit e125556 into main Sep 26, 2025
5 checks passed
@killme2008 killme2008 deleted the feature/static-user-permission branch September 26, 2025 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update docs for feat: supports permission mode for static user provider
3 participants